Skip to content

[schelling] Address editorial items from #419 - #756

Merged
jstac merged 1 commit into
mainfrom
schelling-419-fixes
May 28, 2026
Merged

[schelling] Address editorial items from #419#756
jstac merged 1 commit into
mainfrom
schelling-419-fixes

Conversation

@jstac

@jstac jstac commented May 28, 2026

Copy link
Copy Markdown
Contributor

Addresses concrete, in-scope editorial items from #419.

Changes

  • Rename "Jump Chain Algorithm" → "Move algorithm" — the algorithm (draw a location, accept if happy, otherwise retry) isn't a Markov jump chain; the old title was misleading.
  • Wrap the main-loop pseudocode in a prf:algorithm admonition — was code-block:: none, inconsistent with the earlier algorithm block.
  • Remove print('Entering loop ', count) from run_simulation — produced noisy Entering loop 1, Entering loop 2, … output during simulation.
  • Add legends to both scatter plots — clarifies orange = Type 0, green = Type 1.
  • Move solution-end so the closing prose and the sim_random_select(...) sample run sit inside the dropdown solution (they were invisible until the reader opened it).

Out of scope

The remaining items from #419 are open-ended design suggestions (network diagrams in the preferences section, per-iteration happiness distribution plots, side-by-side figures, namedtuple refactor of location, refocusing the lecture narrative on local interactions). They aren't addressed here.

Closes #419.

🤖 Generated with Claude Code

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[schelling] Editorial Suggestions

1 participant